-
Notifications
You must be signed in to change notification settings - Fork 156
Mcp features ii #771
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mcp features ii #771
Conversation
elements.push(MultimodalElement { | ||
m_type: mime_type, | ||
m_content: image_content.data, | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you see a call actually produce an image?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, mcp puppeteer screenshot, it works
@@ -175,6 +176,9 @@ async fn main() { | |||
crate::git::checkpoints::init_shadow_repos_if_needed(gcx_clone).await; | |||
}); | |||
|
|||
// Start or connect to mcp servers | |||
let _ = running_integrations::load_integrations(gcx.clone(), cmdline.experimental, &["**/mcp_*".to_string()]).await; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, should help with benchmarks
c5fbe79
to
d40a692
Compare
@olegklimov not with the split between mcp sse and mcp stdio (UI is not ready yet, but LSP looks like it is) |
…mand is specified, to match config with claude desktop
5438d76
to
d4f0312
Compare
d4f0312
to
072f0f9
Compare
No description provided.